Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

great fesom cmake changes from @wdeconinck to production_DE #526

Merged

Conversation

wdeconinck
Copy link
Contributor

This is the same as production_DE but rebased on refactoring, and with cmake refactoring applied (as in #515 ).

The extra changes on top of production_DE fesom to be part of ifs-bundle and be automatically detected via find_package(fesom) within ifs-source.

Because this branch is production_DE is rebased on refactoring it doesn't cleanly integrate with production_DE and production_DE should be git reset --hard to this branch.

I have ifs-bundle and ifs-source branches for the DE climateDT lined up once production_DE branch is morphed into this one.

Ulf 'Tiggi' Tigerstedt and others added 30 commits October 19, 2023 13:34
Bugfixes to parms,  widely compatable FindNETCDF, and new envs for atos and lumi
…module.F90 io_netcdf_file_module.F90 from razvan_nextgems_multio
…insic module ISO_FORTRAN_ENV is incompatible with option -fdefault-real-8
@suvarchal suvarchal changed the base branch from refactoring to production_DE October 31, 2023 20:07
@suvarchal suvarchal changed the title production_DE cmake changes to production_DE Oct 31, 2023
@suvarchal suvarchal changed the title cmake changes to production_DE great fesom cmake changes to production_DE Oct 31, 2023
@suvarchal suvarchal changed the title great fesom cmake changes to production_DE great fesom cmake changes from @wdeconinck to production_DE Oct 31, 2023
@suvarchal suvarchal merged commit c4f83fb into FESOM:production_DE Oct 31, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants